Skip to content

Conversation

@silverwind
Copy link
Member

@silverwind silverwind commented Nov 6, 2025

Fixes: #35913
Fixes: #35942

Contains a number of minor CSS fixes.

Fix missing border on targeted speech bubble
Screenshot 2025-11-06 at 22 43 31

Add padding to inline comments, slightly more padding around emoji button
Screenshot 2025-11-06 at 22 38 39

Center text on header in code search results
Screenshot 2025-11-06 at 22 08 01

Tweak emoji selector, reducing font size primarily
Screenshot 2025-11-06 at 22 29 46

Minor tweaks to repo sidebar, reduce font size by 1px, center "Release" text with label.
image

Fix issue comment buttons being misaligned on mobile
Screenshot 2025-11-06 at 22 50 19

Add highlight to actions re-run icon
Screenshot 2025-11-06 at 23 04 30

Fix actions re-run button overflow
image

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 6, 2025
@github-actions github-actions bot added modifies/templates This PR modifies the template files modifies/frontend labels Nov 6, 2025
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Nov 6, 2025
@silverwind
Copy link
Member Author

One more fix added to improve the issue timeline:

Before (content breaks out of box):
Screenshot 2025-11-07 at 13 24 18

After (content is wrapped and slightly more compact):
Screenshot 2025-11-07 at 13 23 57

@wxiaoguang

This comment was marked as resolved.

@wxiaoguang

This comment was marked as resolved.

@wxiaoguang wxiaoguang marked this pull request as draft November 7, 2025 12:52
@wxiaoguang

This comment was marked as resolved.

@silverwind
Copy link
Member Author

This damn history is so finicky, thanks for catching.

@wxiaoguang
Copy link
Contributor

This damn history is so finicky, thanks for catching.

Because there are so many hacky patches, unclear names, style conflicts.

@silverwind
Copy link
Member Author

silverwind commented Nov 7, 2025

Yes, it needs a redesign. I've reverted part of the changes, now it's only doing the wrapping (padding so it does not wrap exactly at element right side):

image

Line height is way too much here but I see no easy fix. The old code never assumed text to wrap.

@wxiaoguang
Copy link
Contributor

Line height is way too much here but I see no easy fix. The old code never assumed text to wrap.

Interesting, the regressions are all related to that problematic "line-height" 🤣

@silverwind silverwind marked this pull request as ready for review November 18, 2025 19:56
@silverwind
Copy link
Member Author

All comments are addressed.

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Nov 19, 2025
@wxiaoguang wxiaoguang enabled auto-merge (squash) November 19, 2025 02:34
@wxiaoguang wxiaoguang merged commit 1da1e64 into go-gitea:main Nov 19, 2025
25 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Nov 19, 2025
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Nov 19, 2025
Fixes: go-gitea#35913
Fixes: go-gitea#35942

Contains a number of minor CSS fixes.

---------

Signed-off-by: silverwind <[email protected]>
Co-authored-by: wxiaoguang <[email protected]>
@GiteaBot GiteaBot added the backport/done All backports for this PR have been created label Nov 19, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull request Nov 19, 2025
* giteaofficial/main:
  Misc CSS fixes (go-gitea#35888)
  Make composer registry support tar.gz and tar.bz2 and fix bugs (go-gitea#35958)
  Change project default column icon to 'star' (go-gitea#35967)
@silverwind silverwind deleted the misccss branch November 19, 2025 07:16
silverwind added a commit that referenced this pull request Nov 19, 2025
Backport #35888 by @silverwind

Fixes: #35913
Fixes: #35942

Contains a number of minor CSS fixes.

Fix missing border on targeted speech bubble
<img width="158" height="90" alt="Screenshot 2025-11-06 at 22 43 31"
src="https://github.com/user-attachments/assets/94696191-353a-4782-a998-2a3d5552ab71"
/>

Add padding to inline comments, slightly more padding around emoji
button
<img width="823" height="301" alt="Screenshot 2025-11-06 at 22 38 39"
src="https://github.com/user-attachments/assets/3ed8f113-13d2-4fad-9d12-81a670540e0b"
/>

Center text on header in code search results
<img width="1328" height="295" alt="Screenshot 2025-11-06 at 22 08 01"
src="https://github.com/user-attachments/assets/41e3d279-8504-4435-9347-e9b969cdfaa2"
/>

Tweak emoji selector, reducing font size primarily
<img width="251" height="219" alt="Screenshot 2025-11-06 at 22 29 46"
src="https://github.com/user-attachments/assets/e892646e-129f-44fd-8333-7a8e14863f03"
/>

Minor tweaks to repo sidebar, reduce font size by 1px, center "Release"
text with label.
<img width="390" height="586" alt="image"
src="https://github.com/user-attachments/assets/397dc36b-11e6-42df-bcdf-e97f4280a90e"
/>

Fix issue comment buttons being misaligned on mobile
<img width="757" height="160" alt="Screenshot 2025-11-06 at 22 50 19"
src="https://github.com/user-attachments/assets/1609d104-1bfe-4913-bfa0-6f4739716d61"
/>

Add highlight to actions re-run icon
<img width="116" height="106" alt="Screenshot 2025-11-06 at 23 04 30"
src="https://github.com/user-attachments/assets/0daed0ac-ef59-432a-b534-b4256d38393b"
/>

Fix actions re-run button overflow
<img width="214" height="125" alt="image"
src="https://github.com/user-attachments/assets/d5c79ce6-ad31-4c06-a411-d79eefb72d02"
/>

Signed-off-by: silverwind <[email protected]>
Co-authored-by: silverwind <[email protected]>
Co-authored-by: wxiaoguang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/done All backports for this PR have been created backport/v1.25 lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/frontend modifies/templates This PR modifies the template files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong ui around the comment button The text of the "Re-run all jobs" button is out of bounds on mobile devices.

4 participants